Skip to content

[GHSA-hcjr-322h-429r] Improper Handling of URL Encoding (Hex Encoding)... - #8960

Open
anderruiz wants to merge 1 commit into
anderruiz/advisory-improvement-8960from
anderruiz-GHSA-hcjr-322h-429r
Open

anderruiz wants to merge 1 commit into
anderruiz/advisory-improvement-8960from
anderruiz-GHSA-hcjr-322h-429r

Conversation

@anderruiz

Copy link
Copy Markdown

@github-actions
github-actions Bot changed the base branch from main to anderruiz/advisory-improvement-8960 August 3, 2026 11:39
@yusuke-koyoshi

Copy link
Copy Markdown

Thanks for this PR. Based on the fix commits and the binary artifacts published to Maven Central, I'd suggest the following changes.

Correct the affected Maven packages and add patched versions.

  • Remove the 8.5.x entries. The issue is the URLDecoder.decode() call on the rewritten URI in RewriteValve, which the fix replaces with UDecoder.URLDecode(). That call does not exist in 8.5.x (checked up to 8.5.100). It was introduced by "Better handling of URLs with literal ';' and '?'" (9.0.x: apache/tomcat@ee3ab54 , first in 9.0.103; 10.1.x: apache/tomcat@066bf6b , first in 10.1.40; 11.0.x: apache/tomcat@fbecc91 , first in 11.0.6).
  • Lower bounds of the remaining entries follow the Apache advisory (11.0.0-M1 / 10.1.0-M1 / 9.0.0.M1). Note that releases before 9.0.103 / 10.1.40 / 11.0.6 do not contain this code path.
  • Add org.apache.tomcat:tomcat (binary distribution), which ships RewriteValve.
  • Add patched versions 11.0.24 / 10.1.57 / 9.0.120 (the entries in this PR only have an upper bound).

Suggested affected products:

Ecosystem Package name Affected versions Patched versions
Maven org.apache.tomcat:tomcat-catalina >= 11.0.0-M1, <= 11.0.23 11.0.24
Maven org.apache.tomcat:tomcat-catalina >= 10.1.0-M1, <= 10.1.56 10.1.57
Maven org.apache.tomcat:tomcat-catalina >= 9.0.0.M1, <= 9.0.119 9.0.120
Maven org.apache.tomcat.embed:tomcat-embed-core >= 11.0.0-M1, <= 11.0.23 11.0.24
Maven org.apache.tomcat.embed:tomcat-embed-core >= 10.1.0-M1, <= 10.1.56 10.1.57
Maven org.apache.tomcat.embed:tomcat-embed-core >= 9.0.0.M1, <= 9.0.119 9.0.120
Maven org.apache.tomcat:tomcat >= 11.0.0-M1, <= 11.0.23 11.0.24
Maven org.apache.tomcat:tomcat >= 10.1.0-M1, <= 10.1.56 10.1.57
Maven org.apache.tomcat:tomcat >= 9.0.0.M1, <= 9.0.119 9.0.120

Evidence:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants